Skip to content

[Core] fix group image send#106

Merged
NoirHare merged 3 commits into
mainfrom
redmomn/fix-group-img
Apr 6, 2026
Merged

[Core] fix group image send#106
NoirHare merged 3 commits into
mainfrom
redmomn/fix-group-img

Conversation

@Redmomn
Copy link
Copy Markdown
Contributor

@Redmomn Redmomn commented Apr 6, 2026

fix #92

A51DE87EC7E2AD600EB46675C51D5083

Copilot AI review requested due to automatic review settings April 6, 2026 04:32
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

This PR aims to fix rich media (notably group image) sending by distinguishing C2C vs group upload “scene” when constructing NTV2 rich media upload requests.

Changes:

  • Updated NTV2 rich media upload request construction to pass a new C2C/group discriminator (1 vs 2) for image uploads.
  • Applied the same discriminator to record and video upload request construction (including video thumbnail sub-file uploads).

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread Lagrange.Core/Internal/Services/Message/NTV2RichMediaUploadService.cs Outdated
Comment thread Lagrange.Core/Internal/Services/Message/NTV2RichMediaUploadService.cs Outdated
Comment thread Lagrange.Core/Internal/Services/Message/NTV2RichMediaUploadService.cs Outdated
Comment thread Lagrange.Core/Internal/Services/Message/NTV2RichMediaUploadService.cs Outdated
Comment thread Lagrange.Core/Internal/Services/Message/NTV2RichMediaUploadService.cs Outdated
Comment thread Lagrange.Core/Internal/Services/Message/NTV2RichMediaUploadService.cs Outdated
@NoirHare NoirHare merged commit ffdf51a into main Apr 6, 2026
12 checks passed
@NoirHare NoirHare deleted the redmomn/fix-group-img branch April 6, 2026 12:37
Bemly pushed a commit to Bemly/CharonAnchor that referenced this pull request May 3, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

无法发送图片

3 participants